Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Missing DOTNET_ROOT Environment Variable For Nix Devs #1203

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

DocNITE
Copy link
Contributor

@DocNITE DocNITE commented Nov 10, 2024

Description

Added missing environment variable for nix-shell. I don't like everytime write export DOTNET_ROOT="/path/to/folder", so it should be done by post script in shell.nix

@github-actions github-actions bot added the Changes: YML Changes any yml files label Nov 10, 2024
@FoxxoTrystan FoxxoTrystan requested review from a team, VMSolidus, DEATHB4DEFEAT and Pspritechologist and removed request for a team November 10, 2024 15:28
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Nov 10, 2024
@FoxxoTrystan FoxxoTrystan added Priority: 1-Urgent Must be resolved immediately Size: 5-Very Small For especially small issues/PRs labels Nov 10, 2024
@stellar-novas
Copy link
Contributor

Why do you need to set the DOTNET_ROOT variable? I haven't needed to before

@Mnemotechnician
Copy link
Contributor

Why do you need to set the DOTNET_ROOT variable? I haven't needed to before

It's needed if you have a non-standard dotnet installation. I have this variable defined in my /etc/profile (linux).

@stellar-novas
Copy link
Contributor

Why do you need to set the DOTNET_ROOT variable? I haven't needed to before

It's needed if you have a non-standard dotnet installation. I have this variable defined in my /etc/profile (linux).

I'm a nix user and I don't need to set it, that's why I'm confused

shell.nix Outdated Show resolved Hide resolved
@DEATHB4DEFEAT
Copy link
Member

I'm a nix user and I don't need to set it, that's why I'm confused

I needed to set it to use dotnet tools to create a migration for #1164

@DEATHB4DEFEAT DEATHB4DEFEAT merged commit 04cdc14 into Simple-Station:master Nov 11, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files Priority: 1-Urgent Must be resolved immediately Size: 5-Very Small For especially small issues/PRs Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants